Skip to content

Ele 4874 dremio #837

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
haritamar wants to merge 33 commits into from
Closed

Ele 4874 dremio #837

haritamar wants to merge 33 commits into from

Conversation

@haritamar
Copy link
Collaborator

@haritamar haritamar commented Aug 6, 2025

No description provided.

GuyEshdat and others added 25 commits August 4, 2025 16:22
@GuyEshdat
handling column names that are reserved keywords in dremio
6931918
@GuyEshdat
added macros for dremio support
ecfa917
@GuyEshdat
renaming because snapshots is a reserved keyword in dremio
fd066cc
@GuyEshdat
making sure that dremio temp table relation has type=table because ot…
a8a4374 
…herwise the dbt-dremio inner macros fail
@haritamar
Ele 4877 dremio package elementary tests (#836)
c002b47 
* dremio support in anomaly tests - WIP

* git ignore dbt_internal_packages (relevant for fusion but no harm in adding now)

* get_package_database_and_schema - use non-views for dremio

* anomaly tests fixes

* remove space

* add newline

* dremio datediff - different implementation for seconds diff
@GuyEshdat
another dremio reserved keyword
90fe8db
@GuyEshdat
another dremio reserved keyword
6d4eda7
@GuyEshdat
removed the select part of the datadiff functio, it cause errors
95948c3
@GuyEshdat
fixed comment
0412bae
@GuyEshdat
macro for generating schema names of dremio in tests, needed because …
3e251cc 
…dremio schema names are dymainc (depends if model is table or not
@GuyEshdat
first attemp of adding dremio to e2e tests
11976a2
@GuyEshdat
change schema sepearator in dremio integration tests
9a63dba
@GuyEshdat
temporary limiting tests to one test only, for faster debugging
1b63275
@GuyEshdat
not clearing the env if the target is dremio
a9017b4
@GuyEshdat
not clearing the env if the target is dremio
c56ed1e
@GuyEshdat
removed the specific test filter in the test-warehousre workflow
da8e5f2
@GuyEshdat
more dremio reserved keywords
b523e08
@GuyEshdat
setting dremio max relation name
79d8ae1
@GuyEshdat
fixing dremio sources in e2e tests
c4e6494
@GuyEshdat
changed column name in test because its previous name was a reserved …
9d81741 
…keyword in deremio
@GuyEshdat
esacping reserved keywords in column names when inserting rows
489a724
@GuyEshdat
changed column name in tests because value is a reserved keyword in d…
d346398 
…remio
@GuyEshdat
handling db and schema of dremio in e2e tests
bce218f
@haritamar
replaced datediff dremio implementation so we won't use timestampdiff
83489c2
@haritamar
use edr_datetime_to_sql in a couple other places
4f4603e
@linear
Copy link

linear bot commented Aug 6, 2025

ELE-4874 support dremio in OSS

@haritamar haritamar had a problem deploying to elementary_test_env August 6, 2025 21:58 — with GitHub Actions Failure
@haritamar haritamar temporarily deployed to elementary_test_env August 8, 2025 13:43 — with GitHub Actions Inactive
@GuyEshdat GuyEshdat closed this Aug 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@haritamar @GuyEshdat